Skip to content

Tell a watcher when a workspace stops on a question, and what it asked - #86

Merged
freekmurze merged 6 commits into
spatie:mainfrom
stebrl:fix/notify-on-permission-ask
Nov 9, 2026
Merged

freekmurze merged 6 commits into
spatie:mainfrom
stebrl:fix/notify-on-permission-ask

Conversation

@stebrl

@stebrl stebrl commented Nov 6, 2026 •

Copy link
Copy Markdown
Contributor

Notify a watching chat when another workspace blocks on a question or permission prompt. Reconcile the presentation feed first and check stored unanswered questions so automatic approvals and old turns cannot spend the watch.

Include the question and options in the notice and workspace listing. Notices bound and fence the other agent's text, with regression coverage for feed recovery, resolved questions, truncation and marker escaping.

Download the pinned CI tools through the GitHub release API after repeated HTTP 504 responses from the plain asset URLs.

On a runner with a presentation feed every live event is handled inside
the catch-up pass, so the ask notice for notify_when_done, guarded on
that pass, never fired: the watching chat kept waiting on a workspace
that was blocked on the owner. Only an ask from a turn since replaced
is stale, so that is now the one condition, decided in the core.
The CLI sends no description with AskUserQuestion, so workspace_list
showed an empty summary and the notify_when_done notice said only that
a question was asked. Both now carry the question and its options from
the ask's input, the notice inside the same fence as a last message.
@stebrl
stebrl marked this pull request as ready for review November 6, 2026 22:38

@freekmurze freekmurze left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed correctness, concurrency and security. Fixed feed reconciliation and durable pending-question handling, added escaping and truncation regressions, and verified full CI on 1c45271.

@freekmurze
freekmurze merged commit f100d65 into spatie:main Nov 9, 2026
8 checks passed
@freekmurze

Copy link
Copy Markdown
Member

Thanks @stebrl for finding both watcher gaps and contributing the fix. Merged with additional feed-recovery, automatic-approval and security regression coverage.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants